Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reaction Codex Fix for Unstable Ingredients #859

Closed
wants to merge 20 commits into from

Conversation

pitaden
Copy link
Contributor

@pitaden pitaden commented Mar 2, 2024

About The Pull Request

Reaction codex entries now reflect cases where their ingredients change at certain temperatures, such as plasma turning to gas or water freezing into ice.

Also adds a label for when reactions are exothermic/endothermic, so it's more clear how a reaction will affect temperature.

Why It's Good For The Game

Fixes #851, because It's nice to know that when you fill chemistry with plasma trying to make dexalin, it isn't the codex's fault for having the wrong temperatures!

Changelog

🆑
qol: reaction codex entries now show you if a reaction is endo/exothermic, and by how much
fix: reaction codex entries now reflect the temperatures required by its ingredients
/:cl:

Kapu1178 and others added 4 commits February 29, 2024 19:59
there's almost certainly more reagents that would need unstable_temperature. and I still feel silly having that variable only get used by the codex.
@pitaden
Copy link
Contributor Author

pitaden commented Mar 2, 2024

Leaving this as a draft for now - I still need to go over all the reagents to make sure there aren't similar cases I forgot about

@pitaden
Copy link
Contributor Author

pitaden commented Mar 2, 2024

guh. accidental push so I suppose I am going with it now. this branch was rebased to the one in #855

I forgot about results potentially being unstable. now the codex tells you hot nitroglycerin might be unwise!
@pitaden
Copy link
Contributor Author

pitaden commented Mar 2, 2024

the main problem this was meant to solve is being removed - so I'm closing this and opening a fresh PR for thermic info

@pitaden pitaden closed this Mar 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dexalin codex entry shows incorrect reaction temperature
5 participants